Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Multi-Folder Backups #428

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

jtmusselman
Copy link

Description

I added the ability to specify a list of folders to back up, as I wanted to back up the end and nether as well, and large servers may want to back up lobbies or assorted worlds. I also tried setting up compression using the zip module, but I am unsure of why it is not working, any pointers?.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

@jtmusselman jtmusselman marked this pull request as draft November 5, 2024 22:30
Copy link

netlify bot commented Nov 5, 2024

Deploy Preview for lodestone-dashboard canceled.

Name Link
🔨 Latest commit a81ebf3
🔍 Latest deploy log https://app.netlify.com/sites/lodestone-dashboard/deploys/672bbfb5ffc7aa00084d2d11

Copy link

netlify bot commented Nov 5, 2024

Deploy Preview for lodestone-storybook canceled.

Name Link
🔨 Latest commit a81ebf3
🔍 Latest deploy log https://app.netlify.com/sites/lodestone-storybook/deploys/672bbfb54ff07b000858a4df

Comment on lines 61 to 69
if (config.compressBackup) {
try {
await eventStream.emitConsoleOut(`[Backup Macro] Compressing backup...`);
await compress(`${combinedBackupFolder}`, `${instancePath}/${config.backupFolderRelative}/${combinedBackupFolder}.zip`);
await eventStream.emitConsoleOut(`[Backup Macro] Compression completed.`);
} catch (e) {
console.log(`[Backup Macro] Error compressing backup folder:`, e);
}
}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CheatCod this part is my attempt at adding compression

@jtmusselman
Copy link
Author

jtmusselman commented Nov 6, 2024

Compression of backups on instances with spaces in their name does not work because that same space is showing up in the path. If you are ok with it, @CheatCod I would like to create a pr to change that behavior to create instance folders with hyphens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant